Skip to content

Conversation

@a-roberts
Copy link
Contributor

## What changes were proposed in this pull request?
There are two tests in this suite that are particularly flaky on the following hardware:

2x Intel(R) Xeon(R) CPU E5-2697 v2 @ 2.70GHz and 16 GB of RAM, 1 TB HDD

This simple PR increases the timeout times and batch duration so they can reliably pass

## How was this patch tested?
Existing unit tests with the two core box where I was seeing the failures often

There are two tests in this suite that are particularly flaky on the following hardware:

2x Intel(R) Xeon(R) CPU E5-2697 v2 @ 2.70GHz and 16 GB of RAM, 1 TB HDD

This simple PR increases the timeout times and batch duration so they can reliably pass
@SparkQA
Copy link

SparkQA commented Sep 14, 2016

Test build #65367 has finished for PR 15094 at commit 9ddecc6.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@srowen
Copy link
Member

srowen commented Sep 15, 2016

I don't feel strongly about it, but you could write these times in units of seconds now.

@srowen
Copy link
Member

srowen commented Sep 16, 2016

Merged to master

@asfgit asfgit closed this in fc1efb7 Sep 16, 2016
wgtmac pushed a commit to wgtmac/spark that referenced this pull request Sep 19, 2016
**## What changes were proposed in this pull request?**
There are two tests in this suite that are particularly flaky on the following hardware:

2x Intel(R) Xeon(R) CPU E5-2697 v2  2.70GHz and 16 GB of RAM, 1 TB HDD

This simple PR increases the timeout times and batch duration so they can reliably pass

**## How was this patch tested?**
Existing unit tests with the two core box where I was seeing the failures often

Author: Adam Roberts <[email protected]>

Closes apache#15094 from a-roberts/patch-6.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants